Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default beacon overrides #182

Merged
merged 6 commits into from
Jun 29, 2024

Conversation

DaleStan
Copy link
Collaborator

This adds an intermediate level of beacon definitions; more general than the per-recipe-row definitions in the production table, but more specific than the whole-page settings.

It was inspired by my desire to tell YAFC that the rows using IR3's small assemblers (~50 of my recipe rows) would be affected by three beacons, while (most of) the remaining recipe rows would be affected by four.

On the module settings page, it adds this section. This allows you to specify the number of beacons, the beacon, and the beacon module on a per-building basis.

image

"Add building override" lets you select any previously unselected crafter that accepts beacon effects. Left and right clicking open the same dialogs as clicking on the beacon and module settings above.

This is a rebase and minor update (changes to the hint texts) of ShadowTheAge#200. I haven't used it recently (I primarily use it for megabasing), but I'm hoping it will be helpful for TURD beacons and similar. I'm hoping it will also satisfy requests like this one.

@DaleStan DaleStan requested a review from veger June 29, 2024 00:54
@DaleStan DaleStan force-pushed the default-beacon-overrides-ce branch from cdbd5bb to c343bd7 Compare June 29, 2024 02:41
@DaleStan DaleStan force-pushed the default-beacon-overrides-ce branch from c343bd7 to 38a39a1 Compare June 29, 2024 14:59
@shpaass shpaass merged commit 9066a10 into shpaass:master Jun 29, 2024
1 check passed
@DaleStan DaleStan deleted the default-beacon-overrides-ce branch July 1, 2024 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants